Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Collection interval random jittering #552

Merged
merged 1 commit into from
Jan 19, 2016
Merged

Collection interval random jittering #552

merged 1 commit into from
Jan 19, 2016

Conversation

sparrc
Copy link
Contributor

@sparrc sparrc commented Jan 19, 2016

closes #460

@sparrc sparrc force-pushed the collection-jitter branch 2 times, most recently from 645b67c to 1d839f4 Compare January 19, 2016 20:05
@sparrc
Copy link
Contributor Author

sparrc commented Jan 19, 2016

cc @sebito91 @daviesalex

@sparrc
Copy link
Contributor Author

sparrc commented Jan 19, 2016

One thing is different from #460, which is that the jitter is simply a random sleep after interval seconds, meaning that

interval = "10s"
collection_jitter = "1s"

would have the plugin run every 10 to 11 seconds (not 9 to 11 seconds as stated in the feature request). You can obviously get 9 to 11 seconds by doing:

interval = "9s"
collection_jitter = "2s"

@sparrc sparrc force-pushed the collection-jitter branch from 1d839f4 to d3a5cca Compare January 19, 2016 20:22
@sebito91
Copy link
Contributor

LGTM, let's push it and try it out.

@sparrc sparrc merged commit d3a5cca into master Jan 19, 2016
@sparrc sparrc deleted the collection-jitter branch January 19, 2016 20:44
@wrigtim
Copy link

wrigtim commented Jan 22, 2016

Thanks Cameron :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RFC] Randomization for plugin execution
3 participants